Skip to content

feat: harnessed Cache to GraphQL criteria evaluation flow#58

Merged
petruki merged 2 commits intomasterfrom
staging
May 5, 2026
Merged

feat: harnessed Cache to GraphQL criteria evaluation flow#58
petruki merged 2 commits intomasterfrom
staging

Conversation

@petruki
Copy link
Copy Markdown
Member

@petruki petruki commented May 5, 2026

This pull request introduces improvements to configuration retrieval and caching mechanisms, as well as updates dependencies and enhances test reliability. The main focus is on optimizing how configuration data is fetched by leveraging caching, and ensuring tests reflect cache state changes.

Caching and configuration retrieval improvements:

  • Refactored resolveConfigByKey in src/aggregator/resolvers.js to first attempt fetching configuration from cache using getConfigFromCache if caching is enabled, falling back to a database query otherwise. This improves performance and reduces database load.

Testing reliability and cache consistency:

  • Updated tests in tests/client-api-cached.test.js to explicitly refresh the domain cache after changing configuration status, ensuring that test results are consistent with the latest cache state. [1] [2]

Dependency updates:

  • Upgraded the express-rate-limit package from version 8.4.1 to 8.5.0 in package.json to incorporate the latest fixes and features.

@petruki petruki added this to the v1.1.0 milestone May 5, 2026
@petruki petruki self-assigned this May 5, 2026
@petruki petruki added the enhancement New feature or request label May 5, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

@petruki petruki merged commit f07d06b into master May 5, 2026
5 checks passed
@petruki petruki deleted the staging branch May 5, 2026 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant